Skip to content

Conversation

@vdusek
Copy link
Contributor

@vdusek vdusek commented Nov 10, 2025

@github-actions github-actions bot added this to the 127th sprint - Tooling team milestone Nov 10, 2025
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Nov 10, 2025
@vdusek vdusek requested a review from Pijukatel November 10, 2025 17:40
@vdusek vdusek marked this pull request as ready for review November 10, 2025 17:40
@@ -0,0 +1,95 @@
const { createHash } = require('node:crypto');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any plan to import this code? Apart from one env var name, it is a copy from the crawlee repo.

Copy link
Contributor Author

@vdusek vdusek Nov 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The roa-loader is just copied from crawlee repo, no changes from my side.

(The duplication of website-related logic is a separate issue.)

vdusek added a commit to apify/crawlee-python that referenced this pull request Nov 11, 2025
- Set headless mode to True by default to improve performance when
running via "Run on Apify", preventing slow execution and potential
timeout errors.
- Align the Python example more closely with its JavaScript counterpart
to keep the documentation consistent across languages.
- Add `asyncio.run(main())` so the snippet is fully copy-and-play ready.
- Also, a minor update regarding the docs code examples (same as in
apify/apify-sdk-python#673).
@vdusek vdusek merged commit 34e1e23 into master Nov 11, 2025
21 checks passed
@vdusek vdusek deleted the add-runnable-code-blocks branch November 11, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "Run on Apify" button to doc code examples

3 participants